Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to anise 0.4.3 #362

Merged
merged 3 commits into from
Sep 9, 2024
Merged

Upgrade to anise 0.4.3 #362

merged 3 commits into from
Sep 9, 2024

Conversation

ChristopherRabotin
Copy link
Member

Summary

Improvements in the computation of eclipsing lead to significant improvements of the solar radiation pressure modeling when compared to GMAT:

Error accumulated in full sunlight over 24 days : 0.488737 m 	0.000081 m/s
test mission_design::force_models::srp_earth_full_vis ... ok

Error accumulated in ecc+inc MEO (with penumbras) over 24 days : 1.866381 m 	0.000635 m/s

Error accumulated in circular equatorial LEO (with penumbras) over 24 days : 5.989316 m 	0.006456 m/s
test mission_design::force_models::srp_earth_leo ... ok

Note that some of these changes could be due to the improvements in the ephemeris itself by using ANISE in the first place, but I don't think so because I'm pretty sure I had checked that.

Architectural Changes

  • EclipseState is gone
  • line_of_sight and eclipse_state are both removed in favor of ANISE's implementation
  • Ruggiero "max eclipse" initialization now takes a penumbra percentage instead of an EclipseState

New Features

No change

Improvements

No change

Bug Fixes

No change

Testing and validation

No additional validation, but rechecking the SRP validation.

Documentation

This PR does not primarily deal with documentation changes.

@ChristopherRabotin ChristopherRabotin merged commit 923aca9 into master Sep 9, 2024
4 of 5 checks passed
@ChristopherRabotin ChristopherRabotin deleted the deps/anise-0.4.3 branch September 9, 2024 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant